bug analyze
Compare with APIPlugin.js
and ExtendedAPIPlugin.js
, we will find this line:
ExtendedAPIPlugin.js#L351
compilation.mainTemplate.plugin("global-hash", () => true);
so, useChunkHash
fn always return false
MainTemplate.js#L2291
2
3
4useChunkHash(chunk) {
const paths = this.applyPluginsWaterfall("global-hash-paths", []);
return !this.applyPluginsBailResult("global-hash", chunk, paths); // always return `false`
}
then noChunkHash
always true
Compilation.js#L12661
2
3
4file = this.getPath(filenameTemplate, {
noChunkHash: !useChunkHash,
chunk
});
then asset-path
plugin will throw the error1
Cannot use [chunkhash] for chunk in '${path}' (use [hash] instead)
TemplatedPathPlugin.js#L631
2
3
4// replacePathVariables
if(data.noChunkHash && REGEXP_CHUNKHASH_FOR_TEST.test(path)) {
throw new Error(`Cannot use [chunkhash] for chunk in '${path}' (use [hash] instead)`);
}
by commenting out the following line in the ExtendedAPIPlugin.js#L35, it works fine.
hack fix:
create
ExtendedAPIPlugin.js
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47/*
MIT License http://www.opensource.org/licenses/mit-license.php
Author Tobias Koppers @sokra
*/
;
const ConstDependency = require("webpack/lib/dependencies/ConstDependency");
const ParserHelpers = require("webpack/lib/ParserHelpers");
const NullFactory = require("webpack/lib/NullFactory");
const REPLACEMENTS = {
__webpack_hash__: "__webpack_require__.h", // eslint-disable-line camelcase
__webpack_chunkname__: "__webpack_require__.cn" // eslint-disable-line camelcase
};
const REPLACEMENT_TYPES = {
__webpack_hash__: "string", // eslint-disable-line camelcase
__webpack_chunkname__: "string" // eslint-disable-line camelcase
};
class ExtendedAPIPlugin {
apply(compiler) {
compiler.plugin("compilation", (compilation, params) => {
compilation.dependencyFactories.set(ConstDependency, new NullFactory());
compilation.dependencyTemplates.set(ConstDependency, new ConstDependency.Template());
compilation.mainTemplate.plugin("require-extensions", function(source, chunk, hash) {
const buf = [source];
buf.push("");
buf.push("// __webpack_hash__");
buf.push(`${this.requireFn}.h = ${JSON.stringify(hash)};`);
buf.push("");
buf.push("// __webpack_chunkname__");
buf.push(`${this.requireFn}.cn = ${JSON.stringify(chunk.name)};`);
return this.asString(buf);
});
// compilation.mainTemplate.plugin("global-hash", () => true);
params.normalModuleFactory.plugin("parser", (parser, parserOptions) => {
Object.keys(REPLACEMENTS).forEach(key => {
parser.plugin(`expression ${key}`, ParserHelpers.toConstantDependency(REPLACEMENTS[key]));
parser.plugin(`evaluate typeof ${key}`, ParserHelpers.evaluateToString(REPLACEMENT_TYPES[key]));
});
});
});
}
}
module.exports = ExtendedAPIPlugin;import the custom plugin in
webpack.config.prod.js
1
2
3
4
5
6const ExtendedAPIPlugin = require('./ExtendedAPIPlugin');
// ...
plugins: [
// ...
new ExtendedAPIPlugin()
]